Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show all descriptions and alignments with maximum score #38

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

krdav
Copy link

@krdav krdav commented Aug 15, 2023

Clunky way of solving the request that I wrote about in this issue:
#37

Not suggesting this as the actual solution, but rather showing it so that it is clear what I am referring to in the issue. In a better solution the show_best option should probably be a boolean or yes/no and there is probably a more intelligent way of implementing it such that showhits and showalignments are not forced to be the same.

@krdav
Copy link
Author

krdav commented Aug 16, 2023

Update:
Instead of showing only top scoring sequences, I changed it such that it allows showing multiple levels e.g.
show_best=1 shows sequences with the highest score
show_best=2 shows sequences with the highest and second highest score
etc

@torognes
Copy link
Owner

Thanks for the suggested change to swipe. I haven't worked on the swipe code for a long time, but will try to find time to look at this soon.

@krdav
Copy link
Author

krdav commented Aug 21, 2023

Thank you so much!
I understand if you got other priorities, not working on the swipe code.
But Swipe just happen to be very fast - in my experience a lot faster than other SW tools/libraries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants